-
Notifications
You must be signed in to change notification settings - Fork 82
IBX-7912: Describe automated translation feature #2414
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thank you! One small remark about the special handling of landing pages
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMHO we can add full synopsis here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mnocon Please take over.
Preview of modified Markdown: |
5fe868e
to
5577c12
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two small remarks.
Co-authored-by: julitafalcondusza <[email protected]>
Co-authored-by: Tomasz Dąbrowski <[email protected]>
code_samples/ change report
|
* IBX-7912: Describe automated translation feature * Fix wording * Update the feature description * Tiny wording fixes * Command name fix * Implement reviewer comments * TMP * Added examples * Added missing links * Before review fixes * Fixes before review * Applied suggestions from code review Co-authored-by: julitafalcondusza <[email protected]> * Apply suggestions from code review Co-authored-by: Tomasz Dąbrowski <[email protected]> * Fixed highlight --------- Co-authored-by: Marek Nocoń <[email protected]> Co-authored-by: julitafalcondusza <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I found few broken links, I'm fixing them in #2658
Add a description of the automated translation feature.
PHPStan is passing on 4.6 branch, it's failing because of the v5/Symfony 6 changes right now

Preview:
The PR is expanded to include changes from:
I've added examples how to:
Checklist